Skip to content

CI: Fix graphviz installation #171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Apr 26, 2023
Merged

CI: Fix graphviz installation #171

merged 5 commits into from
Apr 26, 2023

Conversation

Smit-create
Copy link
Member

@netlify
Copy link

netlify bot commented Apr 23, 2023 •
edited
Loading

✅ Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit d760b9f
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/6448ba9178a22b0007c8409b
😎 Deploy Preview https://deploy-preview-171--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented Apr 23, 2023 •
edited
Loading

🚀 Deployed on https://6448bcd83678c316d7416cc1--taupe-gaufre-c4e660.netlify.app

@Smit-create
Copy link
Member Author

I tested this by building locally and works for me on macOS.

@Smit-create
Copy link
Member Author

@jstac @mmcky This fixes the issue #170. I preferred using conda to install graphviz as it handles the installation and dependencies beautifully following the instructions given at: https://graphviz.readthedocs.io/en/stable/manual.html#installation. The same had worked for me earlier on my local machine when I got the error. Please have a look at it and let me know if you find any issues to any of the lectures.

@jstac
Copy link
Contributor

jstac commented Apr 24, 2023

Thanks @Smit-create, much appreciated.

@mmcky Are you happy with this? I'll let you merge in case you need to edit the style guide.

@Smit-create Smit-create requested a review from mmcky April 26, 2023 05:46
mmcky
mmcky approved these changes Apr 26, 2023
View reviewed changes
Copy link
Contributor

@mmcky mmcky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @Smit-create this is looking great. I will merge this now.

@mmcky mmcky merged commit f3aa05e into main Apr 26, 2023
@mmcky mmcky deleted the i-170 branch April 26, 2023 06:56
@Smit-create Smit-create mentioned this pull request May 20, 2023
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmcky mmcky mmcky approved these changes

Assignees
No one assigned
Labels
ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Smit-create @jstac @mmcky